Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configure yum proxy if given for updates #16972

Merged

Conversation

bdunne
Copy link
Member

@bdunne bdunne commented Feb 8, 2018

@@ -61,6 +61,7 @@ def attempt_registration
return unless register
attach_products
# HACK: #enable_repos is not always successful immediately after #attach_products, retry to ensure they are enabled.
Copy link
Member

@carbonin carbonin Feb 8, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment applies to the line two below. I think the configure_yum_proxy should be above the comment.

@bdunne bdunne force-pushed the configure_proxy_info_for_yum_updates branch from 48d490c to 5798d83 Compare February 8, 2018 16:11
@miq-bot
Copy link
Member

miq-bot commented Feb 8, 2018

Checked commit bdunne@5798d83 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

@carbonin carbonin self-assigned this Feb 8, 2018
@carbonin carbonin added this to the Sprint 79 Ending Feb 12, 2018 milestone Feb 8, 2018
@carbonin carbonin merged commit 58a6e56 into ManageIQ:master Feb 8, 2018
@bdunne bdunne deleted the configure_proxy_info_for_yum_updates branch February 8, 2018 22:20
simaishi pushed a commit that referenced this pull request Feb 27, 2018
@simaishi
Copy link
Contributor

Euwe backport details:

$ git log -1
commit ab37c8864d9676e70854d39bad056c7185c5ed97
Author: Nick Carboni <ncarboni@redhat.com>
Date:   Thu Feb 8 17:19:55 2018 -0500

    Merge pull request #16972 from bdunne/configure_proxy_info_for_yum_updates
    
    Configure yum proxy if given for updates
    (cherry picked from commit 58a6e5645ac73ec5a6942bf97cdb56977f5fa5c4)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1549627

simaishi pushed a commit that referenced this pull request Mar 1, 2018
@simaishi
Copy link
Contributor

simaishi commented Mar 1, 2018

Fine backport details:

$ git log -1
commit 320ad7ea332100b23a21cc1b22159d9adf06dff6
Author: Nick Carboni <ncarboni@redhat.com>
Date:   Thu Feb 8 17:19:55 2018 -0500

    Merge pull request #16972 from bdunne/configure_proxy_info_for_yum_updates
    
    Configure yum proxy if given for updates
    (cherry picked from commit 58a6e5645ac73ec5a6942bf97cdb56977f5fa5c4)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1549626

simaishi pushed a commit that referenced this pull request Mar 7, 2018
@simaishi
Copy link
Contributor

simaishi commented Mar 7, 2018

Gaprindashvili backport details:

$ git log -1
commit e1e13c8d8389a81337f5471da921d5ae64697bb1
Author: Nick Carboni <ncarboni@redhat.com>
Date:   Thu Feb 8 17:19:55 2018 -0500

    Merge pull request #16972 from bdunne/configure_proxy_info_for_yum_updates
    
    Configure yum proxy if given for updates
    (cherry picked from commit 58a6e5645ac73ec5a6942bf97cdb56977f5fa5c4)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1549625

d-m-u pushed a commit to d-m-u/manageiq that referenced this pull request Jun 6, 2018
…r_yum_updates

Configure yum proxy if given for updates
(cherry picked from commit 58a6e56)

https://bugzilla.redhat.com/show_bug.cgi?id=1549626
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants